Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

Boost shift tests #4064

Merged
merged 1 commit into from
Apr 28, 2017
Merged

Boost shift tests #4064

merged 1 commit into from
Apr 28, 2017

Conversation

chfast
Copy link
Member

@chfast chfast commented Apr 26, 2017

No description provided.

// Division equivalence.

// Built-in type:
volatile auto d = int64_min;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this volatile magic for ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing important any more. I wanted to check if runtime results are the same as compile-time results so I put volatile to prevent the compiler to compute it up front. I'm not sure it works.

Copy link
Member

@gumb0 gumb0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, except for the not needed volatile and maybe confusing name for utf alias

@chfast
Copy link
Member Author

chfast commented Apr 28, 2017

I will change both. The utf is suggested by boost docs, but I agree the name is not clear.

@codecov-io
Copy link

Codecov Report

Merging #4064 into develop will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #4064      +/-   ##
===========================================
+ Coverage    65.74%   65.77%   +0.02%     
===========================================
  Files          308      307       -1     
  Lines        22778    22831      +53     
===========================================
+ Hits         14976    15017      +41     
- Misses        7802     7814      +12
Impacted Files Coverage Δ
test/unittests/external-dependencies/boost.cpp 100% <100%> (ø) ⬆️
libethereum/Client.h 68.42% <0%> (-5.27%) ⬇️
test/tools/fuzzTesting/fuzzHelper.cpp 31.37% <0%> (-1.97%) ⬇️
libethcore/Transaction.h 62.96% <0%> (-1.33%) ⬇️
libp2p/Host.cpp 72.36% <0%> (-1.06%) ⬇️
libp2p/Common.cpp 52% <0%> (-0.8%) ⬇️
libethcore/Exceptions.h 82.05% <0%> (-0.45%) ⬇️
libethereum/Client.cpp 24.07% <0%> (-0.22%) ⬇️
libethereum/ChainParams.cpp 95.57% <0%> (-0.16%) ⬇️
test/tools/jsontests/BlockChainTestsBoost.cpp 86.36% <0%> (-0.13%) ⬇️
... and 7 more

@chfast chfast merged commit 0725111 into develop Apr 28, 2017
@chfast chfast deleted the boost-shift-tests branch April 28, 2017 13:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants